#169 Deserialize kebab-case #176
Conversation
|
Thanks so much for contributing! I think it would be nice to have one small unit test that verifies the new behavior, so we prevent regressions in the future. There should already be some models you can use in the Tests project. |
|
I was just checking this PR to implement the missing unit test. Could it be that you already fixed this in another commit? Because the edited files in this PR are now using the propertyNameConverter already. ~Cheers |
|
@PhyberApex yes, i seem to recall fixing this when I was writing some extra tests. I had forgotten about this PR so I didn't post a comment. Are you able to see if #169 is fixed for you in the latest pre-release? |
|
Right now I am not able to check this. But if you want me to, I can give it a go on monday and report back! ~Cheers |
Hi there,
fixed the issue. All unit tests are passing. I have added mine and verified the functionality but haven't actually pushed it in because the data and structure is kinda proprietary.